Skip to content

Pre-release 4.2.104 → 4.2.103 — task 4.3 log swap (illuminate/log v13) - #75

Open
agissept wants to merge 1 commit into
pre-release/4.2.103from
pre-release/4.2.104
Open

agissept wants to merge 1 commit into
pre-release/4.2.103from
pre-release/4.2.104

Conversation

@agissept

Copy link
Copy Markdown
Member

Task 4.3 (L2 per-unit): swap illuminate/log → ^13. Stacked di atas #74 (4.2.103 config).

Perubahan

  • composer: illuminate/log replace → require ^13 (v13.33.0; monolog ^3 sudah di floor).
  • v13 ganti Log\Writer imperatif (useDailyFiles/log.setup) dgn LogManager config-driven; Writer→Logger, getMonolog()→getLogger(). v13 Log\{LogManager,Logger,LogServiceProvider} shadow tree fork lewat PSR-4 seed-mechanism.
  • registerCoreContainerAliases: 'log' → Illuminate\Log\LogManager (dari Log\Logger); buang alias BC Log\Writer yang mati.
  • Drop tests/Log/LogWriterTest.php (vendored-behavior: Writer useFiles/getMonolog).

Fork suite 579 hijau.

App matched-pair: dicoding#5913. Commander fork juga dipatch (EventDispatcher Writer→Logger, dbc73c2).

v13 replaces the imperative Log\Writer (useFiles/useDailyFiles + log.setup closure)
with a config-driven LogManager (reads config/logging.php channels; PSR-3). v13
Log\{LogManager,Logger,LogServiceProvider} shadow the fork tree via PSR-4 seed
mechanism; the fork LogServiceProvider (Writer + log.setup) goes dead.

- composer: illuminate/log self.version -> ^13 (v13.33.0; monolog ^3 already on floor).
- registerCoreContainerAliases: 'log' -> Illuminate\Log\LogManager (was Log\Logger);
  drop the dead Log\Writer BC alias (v13 has no Writer).
- Drop tests/Log/LogWriterTest.php (vendored-behavior: old Writer useFiles/getMonolog).

Fork suite 579 green. App must author config/logging.php (matched-pair).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant